Apply exemplar's new CI system#6
Apply exemplar's new CI system#6ednolan merged 1 commit intobemanproject:mainfrom ednolan:enolan_newci1
Conversation
This commit applies the following changes: - Add bemanproject/infra-workflows as a beman-submodule in .github/workflows - Add bemanproject/infra as a beman-submodule in infra/ - Copy over the latest CMakePresets.json file from exemplar, except set the default to C++23 instead of C++20 - Add an empty lockfile.json to make infra/cmake/use-fetch-content.cmake happy - Update the ci_tests.yml file to provide CI configurations for GCC 14-15 and Clang 19-20 - Update the README to add a code coverage badge
ednolan
requested review from
JeffGarland,
changkhothuychung,
neatudarius and
wusatosi
as code owners
| @@ -0,0 +1 @@ | |||
| * @ednolan | |||
There was a problem hiding this comment.
oh wow -- this is a second codeowners file -- didn't know you could do that
There was a problem hiding this comment.
Nope, this codeowners file has no effect here-- I was thinking about seeing if we can remove it because of how confusing it is. It shows up here because it's this codeowners file:
| # MAC OS | ||
| *.DS_Store | ||
|
|
||
| # Editor files |
There was a problem hiding this comment.
we need emacs on the standard set here -- thought I'd put that in exemplar, but this list is much longer than most repos...
|
|
||
| include(FetchContent) | ||
|
|
||
| if(NOT BEMAN_EXEMPLAR_LOCKFILE) |
There was a problem hiding this comment.
do these need to be renamed?
There was a problem hiding this comment.
They don't need to be, but they should be-- these names are bad, but don't cause any conflicts or functional problems
There was a problem hiding this comment.
we should probably open an issue?
There was a problem hiding this comment.
2 participants
This commit applies the following changes: